Skip to content

Adds ExtraFieldCustomizer for ... customizing extra fields#981

Merged
codefromthecrypt merged 3 commits intomasterfrom
extra-field-customizer
Aug 30, 2019
Merged

Adds ExtraFieldCustomizer for ... customizing extra fields#981
codefromthecrypt merged 3 commits intomasterfrom
extra-field-customizer

Conversation

@codefromthecrypt
Copy link
Member

This will allow you to change field setup without accidentally
overriding the trace propagation format.

Like #978, except for extra fields. I realize this dramatically
simplifies our spring 2.5 variant of brave-webmvc-example along the way.

This will allow you to change field setup without accidentally
overriding the trace propagation format.

Like #978, except for extra fields. I realize this dramatically
simplifies our spring 2.5 variant of brave-webmvc-example along the way.
Copy link
Member

@devinsba devinsba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, is there value in one of these for CurrentTraceContext also?

@codefromthecrypt
Copy link
Member Author

you mean for scope decorators? probably. If you are up for it, go for it.. otherwise I'll try a hand later. will be afk a bit tomorrow.

@codefromthecrypt
Copy link
Member Author

#983 for context customizer.. I think that's all of them at this point.

@codefromthecrypt
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants